home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part1 / 5316 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.7 KB  |  95 lines
  1. Path: magnus.acs.ohio-state.edu!csn!ub!newserve!rebecca!rpi!not-for-mail
  2. From: tonyh@tcp.co.uk
  3. Newsgroups: comp.lang.c++,comp.lang.c++.moderated,comp.lang.c.moderated
  4. Subject: Re: Q: Rigorous coding using #if !defined(...) and #include
  5. Date: 2 Feb 1996 23:58:03 -0000
  6. Organization: UWL
  7. Sender: cppmods@netlab.cs.rpi.edu
  8. Approved: herbs@connobj.com
  9. Message-ID: <4eu8eb$npt@netlab.cs.rpi.edu>
  10. References: <4eo1fs$rr3@netlab.cs.rpi.edu>
  11. Reply-To: tonyh@tcp.co.uk
  12. NNTP-Posting-Host: netlab.cs.rpi.edu
  13. X-Original-Date: Fri, 02 Feb 1996 20:50:03 GMT
  14.  
  15. On 31 Jan 1996 15:22:36 -0000 David Carr wrote:
  16.  
  17. > Okay, nuff of the preamble.  The problem is this.  I have 
  18. > another class;
  19. > 
  20. > #if !defined (__OTHER_H)
  21. > #define (__OTHER_H)
  22. > 
  23. > #include "MyClass.h"
  24. > 
  25. > class Other
  26. > {
  27. > 
  28. ...// Uses something in MyClass.h
  29. > };
  30. > 
  31. > #endif
  32. > 
  33. > and in my source for MyClass.cpp, MyClass.h is obviously 
  34. > included.  Now,
  35. > when I try to compile MyClass.cpp, MyClass.h is included and 
  36. > __MYCLASS_H 
  37. > becomes defined.  Then, MyClass.h #includes Other.h, thus 
  38. > defining __OTHER_H.
  39. > Now, Other.h #includes MyClass.h.  As __MYCLASS_H has already 
  40. > been defined,
  41. > the code in MyClass.h is not included.  Returning to Other.h, 
  42. > this code
  43. > references something in MyClass.h (i.e. its class) and a 
  44. > compiler error is
  45. > generated because the symbol/identifier/whatever is unknown.
  46.  
  47. It sounds like either you haven't split up your declarations/definitions
  48. into enough header files or you've got some circular dependencies that
  49. wouldn't work anyway.
  50.  
  51. If you have something like:
  52.  
  53. class A {
  54.   B *p;
  55. public:
  56.   do_something();
  57. };
  58.  
  59. class B {
  60. public:
  61.   void use_a(A *p)
  62.   {
  63.     a->do_something();
  64.   }
  65. };
  66.  
  67. class B needs a definition of class A because it accesses one or more
  68. members. But class A only has a pointer to B, so it only needs to know
  69. that there is some class called B, without knowing its definition ie
  70. change the member to:
  71.  
  72. class B *p;
  73.  
  74. or declare:
  75.  
  76. class B;
  77.  
  78. before A's definition.
  79.  
  80. You can't have two classes that use each other's members unless you do
  81. it through an abstract interface class.
  82.  
  83. -- 
  84. -------------------------------------------------------------------
  85. Tony Houghton                           tonyh@tcp.co.uk 
  86. Using a RiscPC 700                      http://www.tcp.co.uk/~tonyh
  87. -------------------------------------------------------------------
  88. You don't call cancer a 'popular' disease,
  89. so why call Windows a popular operating system?
  90.  
  91.       [ Articles to moderate: mailto:c++-submit@netlab.cs.rpi.edu ]
  92.       [  Read the C++ FAQ: http://www.connobj.com/cpp/cppfaq.htm  ]
  93.       [  Moderation policy: http://www.connobj.com/cpp/guide.htm  ]
  94.       [      Comments? mailto:c++-request@netlab.cs.rpi.edu       ]
  95.